Skip to content

fix(server): add fix for keep-alive connection on http version 1.0#1634

Merged
seanmonstar merged 1 commit intohyperium:masterfrom
lambdasqd:master
Aug 15, 2018
Merged

fix(server): add fix for keep-alive connection on http version 1.0#1634
seanmonstar merged 1 commit intohyperium:masterfrom
lambdasqd:master

Conversation

@lambdasqd
Copy link
Copy Markdown
Contributor

Change behaviour of connection or server response when the request is
version 1.0 and the Connection: keep-alive header is not present.

  1. If the response is also version 1.0, then connection is closed if the
    server keep-alive header is not present.
  2. If the response is version 1.1, then the keep-alive header is added
    when downgrading to version 1.0.

CLOSES: #1614

Change behaviour of connection or server response when the request is
version 1.0 and the Connection: keep-alive header is not present.

1. If the response is also version 1.0, then connection is closed if the
server keep-alive header is not present.
2. If the response is version 1.1, then the keep-alive header is added
when downgrading to version 1.0.

CLOSES: hyperium#1614
@seanmonstar seanmonstar merged commit 1448e40 into hyperium:master Aug 15, 2018
@seanmonstar
Copy link
Copy Markdown
Member

Awesome PR, thanks so much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

HTTP/1.0 and keep-alive bug

2 participants